I also changed some indentation to match rest of the document

Codecov Report

Patch and project coverage have no change.

Comparison is base (782a25b) 72.11% compared to head (174dedd) 72.11%.

@@           Coverage Diff           @@
##             main     #907   +/-   ##
=======================================
  Coverage   72.11%   72.11%           
=======================================
  Files          54       54           
  Lines        3766     3766           
  Branches     2100     2100           
=======================================
  Hits         2716     2716           
  Misses       1048     1048           
  Partials        2        2           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@juuz0 A brother PR should be done in kiwix-tools to upgrade documentation.

@juuz0 Using the term "hash" is wrong (see https://stackoverflow.com/questions/2800187/what-is-it-when-a-link-has-a-pound-sign-in-it#2800195) and actually misleading (because in programming "hash" refers to hashing algoritms).

Current naming is IMO appropriate and we can keep it even if we move the parameters after the pound character.

@kelson42 Thanks, fragment seems a much better name (another resource suggesting its use: https://www.rfc-editor.org/rfc/rfc3986#section-4.2)

I am not sure what you mean by
Current naming is IMO appropriate and we can keep it even if we move the parameters after the pound character.

Do you mean keeping class name as URLSearchParams? It could work but that's a built-in JS interface. We probably shouldn't be naming it same - may use search queries in future where URLSearchParams is needed.

Alternative name could be FragmentParams

A brother PR should be done in kiwix-tools to upgrade documentation

I looked through https://github.com/kiwix/kiwix-tools/blob/main/docs/kiwix-serve.rst
Seems like it doesn't really need any modifications. It doesn't talk about filtering on welcome page (the only target of this PR)

I changed FragmentParams to extend URLSearchParams instead of Map - this way I dont need to reinvent the URL encoding stuff in FragmentParams